Skip to content

Conversation

@legendecas
Copy link
Member

GYP configurations don't work well with variable expansions as the configuration
is merged after the variable expansion.

Refs: #61327 (comment)
Refs: https://github.com/nodejs/gyp-next/blob/main/docs/InputFormatReference.md#processing-order

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/gyp
  • @nodejs/security-wg

@nodejs-github-bot nodejs-github-bot added build Issues and PRs related to build files or the CI. dependencies Pull requests that update a dependency file. needs-ci PRs that need a full CI run. labels Jan 12, 2026
@legendecas legendecas added the request-ci Add this label to start a Jenkins CI on a PR. label Jan 16, 2026
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jan 16, 2026
@nodejs-github-bot
Copy link
Collaborator

@legendecas legendecas added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels Jan 19, 2026
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jan 19, 2026
@nodejs-github-bot nodejs-github-bot merged commit d1e26f6 into nodejs:main Jan 19, 2026
80 of 81 checks passed
@nodejs-github-bot
Copy link
Collaborator

Landed in d1e26f6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. build Issues and PRs related to build files or the CI. dependencies Pull requests that update a dependency file. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants